-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature: ddcommon-net2 crate #766
base: levi/split-net
Are you sure you want to change the base?
Conversation
BenchmarksComparisonBenchmark execution time: 2024-12-04 23:00:05 Comparing candidate commit 3c7a240 in PR branch Found 1 performance improvements and 6 performance regressions! Performance is the same for 44 metrics, 2 unstable metrics. scenario:credit_card/is_card_number/x371413321323331
scenario:credit_card/is_card_number_no_luhn/ 3782-8224-6310-005
scenario:credit_card/is_card_number_no_luhn/x371413321323331
scenario:redis/obfuscate_redis_string
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
BaselineOmitted due to size. |
be8197d
to
55eec29
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## levi/split-net #766 +/- ##
==================================================
- Coverage 70.55% 70.16% -0.40%
==================================================
Files 297 300 +3
Lines 43547 43849 +302
==================================================
+ Hits 30726 30767 +41
- Misses 12821 13082 +261
|
b8c5304
to
dd97fbf
Compare
fd054f7
to
24c38f7
Compare
c665eb4
to
b713817
Compare
fix: install ring CyrptoProvider manually For some reason I do not yet understand, this used to happen automatically but is not happening, and it causes a crashtracker test `crash_tracking_empty_endpoint` to fail because it doesn't have a CryptoProvider.
ec7c92a
to
5abcad4
Compare
8aafbab
to
a801412
Compare
What does this PR do?
This adds the
ddcommon-net2
crate which uses hyper 1, unlikeddcommon-net1
which uses hyper 0.14. Thedatadog-profiling
anddatadog-profiling-ffi
crates are moved to useddcommon-net2
.Motivation
Hyper migration.
Additional Notes
I was hoping that this specific implementation strategy would get rid of a specific linking issue I hit on Windows in another PR. However,
ddcommon-ffi
depends onddcommon-net1
so both net crates are in play. The relationships are a bit different (better, IMO) but it didn't outright fix the issue. I think all crates built together on Windows need to have the same hyper version dependency :'(How to test the change?
Describe here in detail how the change can be validated.